Skip to content

feat(gym-mode): add logs history scope and deduplication controls (#1240)#1256

Merged
rolandgeider merged 6 commits into
wger-project:masterfrom
pankaj-basnet:feat/gym-mode-log-controls-1240
Jul 9, 2026
Merged

feat(gym-mode): add logs history scope and deduplication controls (#1240)#1256
rolandgeider merged 6 commits into
wger-project:masterfrom
pankaj-basnet:feat/gym-mode-log-controls-1240

Conversation

@pankaj-basnet

@pankaj-basnet pankaj-basnet commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

Summary

This PR implements the requested enhancements for "Last Logs" display in Gym Mode, providing users with more control over their reference data while training.

Changes

  • State Architecture: Added logScopeWeeks and showDistinctLogs to GymModeState with persistence via SharedPreferences.
  • Notifier Logic: Extended GymStateNotifier to handle new configuration keys.
  • Provider Refactor: Updated pastExerciseLogsProvider to support filtering by time-range and toggling deduplication.
  • UI Components:
    • Added Log Scope dropdown and Distinct Logs switch to GymModeOptions in StartPage.
    • Added _LogScopeControls inline widget to LogPage for per-set overrides.

Related Issue(s)

Closes #1240

Please check that the PR fulfills these requirements

  • Tests for the changes have been added (for bug fixes / features)
  • Set a 100-character limit to avoid white space diffs (run dart format .)
Screenshot_1783439434 Screenshot_1783439384 Screenshot_1783439451

pankaj-basnet and others added 6 commits July 7, 2026 22:40
- implement persistence for log scope and distinct settings
- refactor(logs): update pastExerciseLogsProvider to support custom scope and deduplication
- feat(ui): add log scope controls to StartPage and LogPage
We now pass clearLogScopeWeeks so that selecting "current routine" actually
resets the scope instead of keeping the previous week count
I'd say the widget is not big enough to be split up (yet)
@rolandgeider rolandgeider merged commit be1c994 into wger-project:master Jul 9, 2026
4 checks passed
@rolandgeider

Copy link
Copy Markdown
Member

merged 💪🏻

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

More control on what "last logs" displays

2 participants